Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

OVA: Change the scanning for ova and ovf files inside the NFS directory #586

Merged
merged 1 commit into from
Sep 11, 2023

Conversation

bkhizgiy
Copy link
Member

@bkhizgiy bkhizgiy commented Sep 6, 2023

With the current change the files will be scanned as followed:

  • For OVA files we will search the zero level (the NFS itself) and first level (the share sub folders).
    For example giving the nfs share is /nfs these files will be selected:

    • /nfs/vm.ova
    • /nfs/dc/vm.ova
  • For extracted OVA files (ovf format) we will search the first level (the NFS sub folders) and second level (the sub folders sub folders).
    For example giving the nfs share is /nfs these files will be selected:

    • /nfs/vm/vm.ovf
    • /nfs/dc/vm1/vm1.ovf

@bkhizgiy bkhizgiy requested a review from ahadas as a code owner September 6, 2023 10:10
@bkhizgiy bkhizgiy requested review from bennyz and liranr23 September 6, 2023 10:10
cmd/ova-provider-server/ova-provider-server.go Outdated Show resolved Hide resolved
cmd/ova-provider-server/ova-provider-server.go Outdated Show resolved Hide resolved
cmd/ova-provider-server/ova-provider-server.go Outdated Show resolved Hide resolved
cmd/ova-provider-server/ova-provider-server.go Outdated Show resolved Hide resolved
cmd/ova-provider-server/ova-provider-server.go Outdated Show resolved Hide resolved
cmd/ova-provider-server/ova-provider-server.go Outdated Show resolved Hide resolved
cmd/ova-provider-server/ova-provider-server.go Outdated Show resolved Hide resolved
@bkhizgiy bkhizgiy force-pushed the path branch 2 times, most recently from 783901d to 6d148f7 Compare September 7, 2023 14:12
cmd/ova-provider-server/ova-provider-server.go Outdated Show resolved Hide resolved
cmd/ova-provider-server/nfs_test.go Outdated Show resolved Hide resolved
cmd/ova-provider-server/nfs_test.go Outdated Show resolved Hide resolved
cmd/ova-provider-server/nfs_test.go Show resolved Hide resolved
@bkhizgiy bkhizgiy force-pushed the path branch 2 times, most recently from 6be073c to a44754b Compare September 10, 2023 17:02
cmd/ova-provider-server/nfs_test.go Outdated Show resolved Hide resolved
cmd/ova-provider-server/nfs_test.go Outdated Show resolved Hide resolved
cmd/ova-provider-server/nfs_test.go Outdated Show resolved Hide resolved
cmd/ova-provider-server/nfs_test.go Outdated Show resolved Hide resolved
@sonarcloud
Copy link

sonarcloud bot commented Sep 11, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 4 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@ahadas ahadas added this to the v2.5.1 milestone Sep 11, 2023
@ahadas ahadas merged commit ea39abb into kubev2v:main Sep 11, 2023
7 checks passed
bkhizgiy added a commit to bkhizgiy/forklift that referenced this pull request Sep 14, 2023
ahadas pushed a commit that referenced this pull request Sep 18, 2023
@ahadas ahadas removed this from the v2.5.1 milestone Sep 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants